[MPOM-296] Remove pmd from Maven site reports#52
[MPOM-296] Remove pmd from Maven site reports#52slawekjaranowski wants to merge 1 commit intoapache:masterfrom slawekjaranowski:MPOM-296
Conversation
slawekjaranowski
requested review from
cstamas,
hboutemy,
michael-o,
olamy and
slachiewicz
|
AFAIK, results from such report should be reviewed: the issue is not in the report but in how the output is not used how much time does it cost (only at release time)? |
|
if we want pmd should be included in build, like checkstyle |
|
I don't see why one would remove the usefulness of the other |
|
ok, I will not fight for it |
|
@olamy is working on such reports during build time on jenkins. Having report here we have view on issues in release time, more of component is release not often so historical view is not usable for me. We should look at commit time if we remove some of issues and more important not introduce new one with next commit, |
hboutemy
left a comment
hboutemy
left a comment
There was a problem hiding this comment.
-1
adding a report to Jenkins for HEAD is probably very interesting in addition to the report in release to stay as a point in time
but don't remove one for the other: both are useful
|
Resolve #254 |
5 participants
No description provided.